Skip to content

feat(core): Adding a fake LLM with tool calling capabilities for easy integration with LangGraph #32002

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

feat(core): Adding a fake LLM with tool calling capabilities for easy integration with LangGraph #32002

wants to merge 3 commits into from

Conversation

sijil-salim
Copy link

Added a new Fake LLM in lines of FakeListChatModel to return tool calls so that it is directly compatible with LangGraph. This fake LLM can be used for testing LangGraph workflows

@sijil-salim sijil-salim requested a review from eyurtsev as a code owner July 13, 2025 10:57
@vercel Vercel
Copy link

vercel bot commented Jul 13, 2025 •
edited
Loading

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Skipped Deployment
Project Deployment Preview Comments Updated (UTC)
langchain ⬜️ Ignored Preview Aug 11, 2025 10:37pm

@sijil-salim
Copy link
Author

@eyurtsev Hi, I pushed a change for adding a fake llm with tool building capabilities yesterday. However, the deployment got skipped and got a message from verbal with links which upon opening shows Vercel error page. Can you please help me with what might have gone wrong and how I can rectify it

@mdrxy mdrxy requested a review from sydney-runkle July 14, 2025 15:24
eyurtsev
eyurtsev requested changes Jul 14, 2025
View reviewed changes
Copy link
Collaborator

@eyurtsev eyurtsev left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Will review a bit more carefully later, but we don't want another implementation. Instead one of the existing implementations should be enhanced properly

@mdrxy mdrxy added the core Related to the package `langchain-core` label Jul 28, 2025
@mdrxy mdrxy changed the title core: Adding a fake LLM with tool calling capabilities for easy integration with LangGraph feat(core): Adding a fake LLM with tool calling capabilities for easy integration with LangGraph Jul 28, 2025
@codspeed-hq CodSpeed HQ
Copy link

codspeed-hq bot commented Jul 28, 2025 •
edited
Loading

CodSpeed WallTime Performance Report

Merging #32002 will not alter performance

Comparing sijil-salim:fake-llm-with-tool-calling (da26492) with master (166c027)

⚠️ Unknown Walltime execution environment detected

Using the Walltime instrument on standard Hosted Runners will lead to inconsistent data.

For the most accurate results, we recommend using CodSpeed Macro Runners: bare-metal machines fine-tuned for performance measurement consistency.

Summary

✅ 13 untouched benchmarks

@codspeed-hq CodSpeed HQ
Copy link

codspeed-hq bot commented Jul 28, 2025 •
edited
Loading

CodSpeed Instrumentation Performance Report

Merging #32002 will not alter performance

Comparing sijil-salim:fake-llm-with-tool-calling (da26492) with master (166c027)

Summary

✅ 14 untouched benchmarks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@eyurtsev eyurtsev eyurtsev requested changes

@sydney-runkle sydney-runkle Awaiting requested review from sydney-runkle

Requested changes must be addressed to merge this pull request.

Assignees

@sydney-runkle sydney-runkle

Labels
core Related to the package `langchain-core`
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@sijil-salim @eyurtsev @sydney-runkle @mdrxy